Skip to content

PYTHON-5713 Allow using OP_MSG exclusively#2799

Draft
sleepyStick wants to merge 3 commits into
mongodb:masterfrom
sleepyStick:PYTHON-5713
Draft

PYTHON-5713 Allow using OP_MSG exclusively#2799
sleepyStick wants to merge 3 commits into
mongodb:masterfrom
sleepyStick:PYTHON-5713

Conversation

@sleepyStick
Copy link
Copy Markdown
Contributor

@sleepyStick sleepyStick commented May 8, 2026

PYTHON-5713

Changes in this PR

  • handshake now always uses OP_MSG (and not OP_QUERY)
  • if server max wire version doesn't support OP_MSG, a ConfigurationError is raised

Test Plan

  • updated tests in test_handshake.py
  • added a new test for the ConfigurationError in test_handshake.py

Checklist

Checklist for Author

  • Did you update the changelog (if necessary)?
  • Is there test coverage?
  • Is any followup work tracked in a JIRA ticket? If so, add link(s).

Checklist for Reviewer

  • Does the title of the PR reference a JIRA Ticket?
  • Do you fully understand the implementation? (Would you be comfortable explaining how this code works to someone else?)
  • Is all relevant documentation (README or docstring) updated?

@sleepyStick sleepyStick mentioned this pull request May 8, 2026
Open
4 tasks
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 8, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 50.00000% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pymongo/asynchronous/pool.py 50.00% 1 Missing and 1 partial ⚠️
pymongo/synchronous/pool.py 50.00% 1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sleepyStick @codecov-commenter